Skip to content

[3.7] bpo-35545: Fix asyncio discarding IPv6 scopes (GH-11271)#13379

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-ac8eb8f-3.7
May 17, 2019
Merged

[3.7] bpo-35545: Fix asyncio discarding IPv6 scopes (GH-11271)#13379
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-ac8eb8f-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented May 17, 2019

Copy link
Copy Markdown
Contributor

This PR proposes a solution to bpo-35545 by adding an optional flowinfo and scopeid to asyncio.base_events._ipaddr_info to carry the full address information into _ipaddr_info and avoid discarding IPv6 specific information.

Changelog entry & regression tests to come.

https://bugs.python.org/issue35545
(cherry picked from commit ac8eb8f)

Co-authored-by: Erwan Le Pape lepaperwan@users.noreply.github.com

https://bugs.python.org/issue35545

This PR proposes a solution to [bpo-35545](https://bugs.python.org/issue35545) by adding an optional `flowinfo` and `scopeid` to `asyncio.base_events._ipaddr_info` to carry the full address information into `_ipaddr_info` and avoid discarding IPv6 specific information.

Changelog entry & regression tests to come.

https://bugs.python.org/issue35545
(cherry picked from commit ac8eb8f)

Co-authored-by: Erwan Le Pape <lepaperwan@users.noreply.github.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@lepaperwan: Status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit 9470404 into python:3.7 May 17, 2019
@miss-islington
miss-islington deleted the backport-ac8eb8f-3.7 branch May 17, 2019 09:05
@miss-islington

Copy link
Copy Markdown
Contributor Author

@lepaperwan: Status check is done, and it's a success ✅ .

@aixtools

Copy link
Copy Markdown
Contributor

FYI: since this was added the AIX bots (and manual testing) fail for test_asyncio.

Shall continue discussion in https://bugs.python.org/issue35545

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants